Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

udx-docker-sftp/master -> icamiami-docker-sftp/master #5

Open
wants to merge 76 commits into
base: master
Choose a base branch
from

Conversation

planv
Copy link

@planv planv commented Nov 15, 2024

No description provided.

planv and others added 30 commits April 6, 2023 22:43
release version 0.2.5
changed gcloud install method
merge 0.2.6 to master
@planv planv requested a review from kavaribes November 15, 2024 19:47
kavaribes and others added 9 commits December 11, 2024 18:52
Updated node modules. Updated Alpine to node:23.4-alpine UDXI-427
updated openssh UDXI-427
Key Changes:
- Add dynamic SFTP server path resolution for different container types
- Consolidate environment variable handling in sshd_config
- Add comprehensive documentation for setup and troubleshooting
- Improve error handling and logging in entrypoint script

Testing Required:
1. SFTP connections with different container types
2. Environment variable passing (TERM, LANG, LC_*)
3. Interactive and non-interactive SSH sessions
4. Error scenarios and logging
5. File transfers (SFTP/SCP)
- Properly order environment variables in sshd_config
- Add logging locations to main README
- Document debug commands for quick troubleshooting
- Replace specific app names with www-myapp-com
- Add realistic pod name examples
- Include example log entries
- Keep naming consistent across docs
- Update client configuration examples
kavaribes
kavaribes previously approved these changes Jan 9, 2025
fix: Improve SFTP reliability and documentation
@kavaribes kavaribes self-requested a review January 12, 2025 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants